-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DNM: Enable the integration tests for targets that were disabled earlier #470
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #470 +/- ##
=======================================
Coverage 34.18% 34.18%
=======================================
Files 143 143
Lines 11040 11040
Branches 2291 2291
=======================================
Hits 3774 3774
Misses 7266 7266
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Build failed. ❌ ansible-test-cloud-integration-vmware-rest FAILURE in 15m 44s |
recheck |
Build failed. ❌ ansible-test-cloud-integration-vmware-rest FAILURE in 15m 28s |
@GomathiselviS I don't know why this fails. Actually, I don't understand the logs at all. Do you have an idea what's wrong here? |
@mariolenz , these tests were disabled previously. I wanted to investigate why they are failing, so I've enabled them for now. I'll look into the issues next week and keep you updated.
|
Yes, that just what I don't understand. I don't know what this error wants to tell me. Well, maybe you find out more. Good luck! |
@alinabuzachis Great! Thanks for having a look at this and find out what's going wrong! Now the question is: What's better / easier, changing the script or renaming |
@GomathiselviS @alinabuzachis Any news on this? |
Recheck |
Build failed. ❌ ansible-test-cloud-integration-vmware-rest RETRY_LIMIT in 12m 41s |
recheck |
Build failed. ❌ ansible-test-cloud-integration-vmware-rest FAILURE in 18m 14s |
I think we should close this since its stale and there are new tests being written to migrate this repo off of zuul. @mariolenz what do you think? |
SUMMARY
Enabling the tests to understand the reason why they were disabled earlier.
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION